Skip to content

feat(auth): add confirm password to signup and settings #1265

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 8, 2015

Conversation

kingcody
Copy link
Member

@kingcody kingcody commented Sep 8, 2015

Originally submitted by @thomporter

Closes #484

@kingcody kingcody force-pushed the feature/confirm-field branch 2 times, most recently from 93578fc to fda8d59 Compare September 8, 2015 02:13
@Awk34
Copy link
Member

Awk34 commented Sep 8, 2015

Perhaps it might be better to do this one without an extra bower package?

@Awk34 Awk34 added this to the 3.1.0 milestone Sep 8, 2015
@kingcody kingcody force-pushed the feature/confirm-field branch from fda8d59 to ac39051 Compare September 8, 2015 04:49
@kingcody
Copy link
Member Author

kingcody commented Sep 8, 2015

That was my thought at first, but honestly its would be the same amount of code for us to implement it ourselves. Also there are a decent amount of stars/forks of that module so I'm sure there will be far more eyes on the code in that repo than there ever would be here on that specific piece.

@kingcody kingcody assigned kingcody and Awk34 and unassigned kingcody Sep 8, 2015
Awk34 added a commit that referenced this pull request Sep 8, 2015
feat(auth): add confirm password to signup and settings
@Awk34 Awk34 merged commit f16cb71 into angular-fullstack:canary Sep 8, 2015
@Awk34
Copy link
Member

Awk34 commented Sep 8, 2015

Alright, sounds good

@kingcody kingcody deleted the feature/confirm-field branch September 9, 2015 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants